feat: Migrate from Java 17 to Java 21 LTS with Spring Boot 3.3.13#184
Open
devin-ai-integration[bot] wants to merge 4 commits intoDevOpsfrom
Open
feat: Migrate from Java 17 to Java 21 LTS with Spring Boot 3.3.13#184devin-ai-integration[bot] wants to merge 4 commits intoDevOpsfrom
devin-ai-integration[bot] wants to merge 4 commits intoDevOpsfrom
Conversation
- Upgrade Java version from 17 to 21 (latest LTS) - Upgrade Spring Boot from 3.3.3 to 3.5.3 - Upgrade maven-compiler-plugin from 3.8.0 to 3.15.0 (source/target 1.8 -> 21) - Replace deprecated mysql:mysql-connector-java with com.mysql:mysql-connector-j - Replace deprecated MySQL8Dialect with MySQLDialect - Update Dockerfile base images to Eclipse Temurin 21 (alpine) Co-Authored-By: Joao Esteves <joao.esteves@cognition.ai>
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
…ability Spring Boot 3.5.3 introduced a transitive dependency with a known vulnerability flagged by Snyk. Spring Boot 3.4.7 is the latest 3.4.x patch release and passes Snyk security checks. Co-Authored-By: Joao Esteves <joao.esteves@cognition.ai>
Use the latest patch release of the same minor version (3.3.13) to minimize transitive dependency changes while still upgrading Java to 21. Co-Authored-By: Joao Esteves <joao.esteves@cognition.ai>
Pin the MySQL connector version to avoid pulling a newer version via the Spring Boot BOM that may trigger Snyk license policy violations. Co-Authored-By: Joao Esteves <joao.esteves@cognition.ai>
devin-ai-integration
Bot
changed the title
Author
End-to-End Test Results — Java 17 → 21 MigrationTested locally with Maven ( Escalations
Results
Shell test evidenceJava 21 runtime confirmed from startup logs: Docker image also runs Java 21: Stack versions: Spring Boot 3.3.13, Hibernate ORM 6.5.3.Final, Apache Tomcat 10.1.42 Browser test screenshotsDashboard after login — "Welcome, testuser", Balance: $0.00 After $500 deposit — Balance: $500.00 After $100 withdrawal — Balance: $400.00 Transaction history — Deposit $500 + Withdrawal $100 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Migrates the BankApp from Java 17 to Java 21 LTS (latest long-term support release) and upgrades all related dependencies to their latest compatible versions.
Changes
mysql:mysql-connector-java:8.0.33com.mysql:mysql-connector-j:8.0.33(updated artifact name)MySQL8Dialect(deprecated)MySQLDialectmaven:3.8.3-openjdk-17maven:3.9-eclipse-temurin-21-alpineopenjdk:17-alpineeclipse-temurin:21-jre-alpineWhy these versions
openjdkDocker images with actively maintained, production-ready builds.mysql:mysql-connector-javaartifact was renamed tocom.mysql:mysql-connector-j— version pinned at 8.0.33 to match the original.maven-compiler-pluginsource/target was inconsistently set to1.8despite the project declaring Java 17 — now correctly aligned to21.Review & Testing Checklist for Human
docker-compose up(rebuild the image withdocker build -t bankapp:java21 .)Notes
mvn clean install -DskipTestspasses).cognition-defaultSnyk org to diagnose the specific issue. This may require an org admin to review the Snyk policy or approve the dependency changes.Link to Devin session: https://app.devin.ai/sessions/56a7d4ebfcb54417b6c23a74a72c0694
Requested by: @joao-cognition
Devin Review